Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat/check status api #19

Merged
merged 20 commits into from
Dec 10, 2024
Merged

Feat/check status api #19

merged 20 commits into from
Dec 10, 2024

Conversation

Draply
Copy link
Member

@Draply Draply commented Nov 26, 2024 •
edited by andrew2003
Loading

Check List

  • Don't forget to squash commits into meaningful chunks before merging
  • Check every test passed.
  • Did you split imports into std, third parties, and custom parts?
  • Format your code.
    • cargo fmt
    • cargo clippy
    • taplo fmt --config taplo/taplo.toml
  • Check unwrap, expect().
  • Check clone().
  • Check your commit messages.
  • Have you added meaningful comments.

@zk-steve zk-steve changed the base branch from master to feat/server November 26, 2024 07:57
zk-steve
zk-steve reviewed Nov 26, 2024
View reviewed changes
Cargo.toml Outdated Show resolved Hide resolved
zk-steve
zk-steve reviewed Nov 29, 2024
View reviewed changes
crates/public/src/main.rs Outdated Show resolved Hide resolved
zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
crates/public/src/utils.rs
#[test]
fn test_save_cairo_pie() {
let encoded_origin = encode_zip("./src/assets/test_data/fibonacci_with_output.zip");
let written_path = save_cairo_pie(&encoded_origin, "test").unwrap();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

write the file to a Tmp folder

zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
Dockerfile Show resolved Hide resolved
zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
Dockerfile Show resolved Hide resolved
zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
crates/public/src/tests/test_add_job.rs
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rewrite this, reference: https://github.com/zk-steve/frog/blob/main/e2e-tests/tests.rs

zk-steve
zk-steve reviewed Dec 9, 2024
View reviewed changes
Cargo.toml Outdated Show resolved Hide resolved
@andrew2003 andrew2003 merged commit 423d97c into feat/server Dec 10, 2024
3 checks passed
@zk-steve zk-steve deleted the feat/check_status_api branch December 19, 2024 04:07
zk-steve added a commit that referenced this pull request Dec 19, 2024
@andrew2003 @hduoc2003
@zk-steve @Draply @VanhGer
feat: add server, worker, rewrite (#23)
0cf8212 
* feat: add function verify_proof_and_register

* fix: flow_test

* chore: more info about transactions

* feat: optimize verify_proof_and_register

* fix: tx

* temp

* feat: new-verifier-contracts

* update test

* feat: add function verify_proof_and_register (#5)

* feat: add function verify_proof_and_register

* fix: flow_test

* chore: more info about transactions

* feat: optimize verify_proof_and_register

* fix: tx

* fix(CI): increase timeout for tarpaulin

---------

Co-authored-by: Steve Nguyen <[email protected]>
Co-authored-by: draply <[email protected]>

* feat: add proof adapter

* feat: init stone-prover-cli

* feat: add proof generator func

* refactor: init the workspace

* refactor: change the src code into deployer subcrate.

* feat: add errors handler for each subcrate

* feat: Add verifier subcrate, which sends proof to the contracts.

* feat: server template

* temp

* feat: add worker service

* feat: add subcrates

* refactor: rename packages

* feat: add worker template

* Feat/check status api (#19)

* feat: add check status api

---------

Co-authored-by: AndrewTran <[email protected]>
Co-authored-by: Andrew Tran <[email protected]>

* feat: update README.md

* sot-190-add_job-worker-handler (#20)

* feat: handle stone verify job

* chore: resolve conflict

* feat: update README.md

* chore: clean code

---------

Co-authored-by: hduoc2003 <[email protected]>

* feat: SOT 206 Add e2e-tests (#22)

* feat: handle stone verify job

* chore: resolve conflict

* feat: add e2e-test

* fix: fix config test

* fix: fix worker not migration

---------

Co-authored-by: Draply <[email protected]>

---------

Co-authored-by: hduoc2003 <[email protected]>
Co-authored-by: Steve Nguyen <[email protected]>
Co-authored-by: draply <[email protected]>
Co-authored-by: hduoc2003 <[email protected]>
Co-authored-by: vanhger <[email protected]>
Co-authored-by: Draply <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew2003 andrew2003 andrew2003 left review comments

@zk-steve zk-steve zk-steve approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Draply @zk-steve @andrew2003